togglebutton: Move :inconsistent to GtkCheckButton

togglebutton: Move :draw-indicator property to GtkCheckButton

It's not used in GtkToggleButton at all, only in GtkCheckButton and
GtkRadioButton.

checkbutton: Remove unused vfunc

checkbutton: Remove unused function prototype

widget: Remove gtk_widget_class_list_style_properties

Unused.

gtkbuilder: Remove unused struct

build: Stop compressing ui files

It's slower at runtime. Instead, strip blanks.

mir: handle surface placement events

This allows the Mir backend to properly emit "moved-to-rect."

mir: use mir_surface_spec_set_placement for menus

This API was added to Mir for GTK menus, combo boxes, etc.

Update Galician translation

Update Galician translation

Update Slovak translation

Update Slovak translation

Remove tests/testtreemenu

Since GtkTreeMenu became a private class only used by GtkComboBox, all
this test actually did was to show a ComboBox constructed with a custom
CellArea. Now that the latter is no longer possible, the test just shows
a handful of settings that do nothing. Just test GtkComboBox directly.

Fix translator comments

They need to be exactly one line above a string to show up in .po files.

Revert "notebook: redraw arrows after closing a tab"

This reverts commit 218beb2ef49d407b547047693605328c10d4877d.

wayland: Ensure to set gtk_surface dbus properties after hide()

Currently hiding destroys the wl_surface and all related interfaces,
(including the gtk_surface1) so the next time the GdkWindow is mapped,
we don't bother to set the DBus properties. Toggle the check off so
it's actually issued again after the GdkWindow gets a gtk_surface1.

https://bugzilla.gnome.org/show_bug.cgi?id=773686

Update Kazakh translation

Update Kazakh translation

notebook: redraw arrows after closing a tab

https://bugzilla.gnome.org/show_bug.cgi?id=777547

Update German translation

widget: Remove gtk_widget_get_preferred_size mention from docs

Mention gtk_widget_measure instead.

flowbox: Measure child widget sizes using gtk_widget_measure

widget: Remove warning about already visible widgets

That was just added by mistake of course.

Remove unnecessary gtk_widget_show calls

gtkfontbutton: hexpand the font label

To restore the previous look.

3to4: Fix typo in previous commit

3to4: Fix suggested replacements for border-width

The new CSS border-spacing does what Grid::(row|column)_spacing and
Box::spacing already did, i.e. controlling the space added between child
widgets, so it’s not a replacement for Container::border-width.

combobox: Stop pointlessly NULL-checking the menu

We now have a valid menu to use at all times.

combobox: Move area and menu to .ui file

Now that priv->area is guaranteed to be constructed by us, and not
passed in by a user, we can move it to the .ui file and stop manually
managing its lifetime altogether. And once the area is there, we can
move the menu there too (and stop pointlessly destroying/rebuilding it).

combobox: Remove property cell-area

The CellArea is going to become a pure implementation detail & be moved
to the .ui, instead of letting users mess with it (if anyone ever did).

combobox: Reuse code rather than reimplementing it

We already have cell_layout_is_sensitive() to get whether at least one
cell in a Layout is sensitive, which we need because CellLayout/View
do not implement foreach(). So, since we wrote that, we can use it to
check our CellArea too, instead of doing foreach with a custom callback.

combobox: some code cleanups

 * Merge adjacent if statements with the same condition
 * Add a space between a function and its declaration

menushell: Fix select_last() to get last, not 1st

It was broken in the same way as select_first(), as explained in
commit a5211adb225172195d6778d652b58632c38afcbf

combobox: Fix documentation typo bx => box

menushell: Make select_first() do that, not last…

It was looping over all items, not breaking out when it found the first
selectable one, and then selecting the _last_ selectable one (if any)
found. So, it did exactly the opposite of its name. This made me quite
baffled when opening a submenu with right-arrow put me at its last item.

Originally, the loop set to_select and broke if the current item was
selectable and not tear-off, meaning that it would correctly select the
first suitable item. However, when tear-off functionality was removed
in commit 4ed9452e9028bb90589474783cebac57ab66e2c9, so was the break.

Update Polish translation

Fix CSS spelling in a translatable string

menushell: Don’t dereference before typechecking

Update POTFILES.in and POTFILES.skip

combobox: If typechecking, don’t reference before

Don't hide windows and popovers in init()

Instead, don't make them visible in gtk_widget_init in the first place.

fontbutton: Be a GtkWidget

assistant: Don't forget to remove child widgets

toolitemgroup: Properly remove internal child widget

We now need to use gtk_widget_unparent and can't just destroy it.

popovermenu: Remove forall implementation

button: Don't explicitly show child widgets

recorder: Adapt to widget visibility changes

We now need to explicitly hide the details label.

Fix some widget visibility fallout

widget: Remove show-all property

Doesn't make sense anymore now that gtk_widget_show_all is gone.

Remove gtk_widget_show_all

widget: Remove show_all vfunc

filechooserbutton: Be a GtkWidget

Instead of subclassing GtkBox to have 2 child widgets, subclass
GtkWidget and only render one of them.

sizerequest: Use gtk_widget_measure to measure widget sizes

gtkbuilder: Improve error messages

shortcutswindow: Unset search_bar in dispose

And check for null-ness in unmap. This prevents a crash when the window
is first disposed and then unmapped.

shortcutswindow: Use GtkButton:icon-name for search button

shortcutsshortcut: Be a GtkWidget

Instead of subclassing GtkBox and overriding _add to not work, subclass
GtkWidget and add a normal GtkBox as a child widget.

gtkshortcutswindow: Adapt to GtkWidget visibility changes

widget-factory: Remove some gtk3-widget-factory leftovers

filechooser: adapt to GtkWidget:visible changes

widget-factory: Adapt to GtkWidget visibility changes

Make the infobar and the toolbar invisible by default and remove all the
lines showing the other widgets.

coloreditor: Hide overlay boxes

Hide popovers and toplevels by default

widget: Be visible by default

combobox: popdown() the menu during unmap()

combo_box_popdown() currently skips popping down our menu if it is NULL.
But the required call to this at end-of-life was in destroy(), by which
point dispose() already NULLed the menu, so Menu::popdown() would never
run, even if it should. Fix this by trying popdown() earlier in unmap().
Also, add a converse assurance that we don’t popup() while not mapped.

Even once we remove all the now-pointless NULL checks, destroy() was the
wrong place to call combo_box_popdown(), and unmap() is the right place.

Revert "wayland: Push NULL buffer when hiding a GdkWindow"

This reverts commit 6beb0b91c91dd4a018b0fe2c39f5687a00938821.

update zh_CN translation

gtkmain: Fix build on Windows

gtk_init() removed its support for supporting arguments, so we ought to do
likewise for Windows, which actually defines items that call gtk_init()
the old way (and also get rid of argument support in those functions,
since the direction is to not support them).

https://bugzilla.gnome.org/show_bug.cgi?id=773299

Visual Studio builds: Update GSK project

Another GSK Vulkan renderer source was added for the build, so include it
here.

gtk: Register GSK resources at startup

That way they show up in the resource list in the inspector even when
running with the Cairo renderer.

combobox: Remove dead code & unneeded type checks

Commit fdc0c6426b6fb751653f3a8536a344f69a6407ac removed the appears-as-
list style property, & hence the ability to put the ComboBox into list
mode – but it left behind a pile of hijinks that were only used in said
mode & so were now doing absolutely nothing. This commit deletes those.

While doing that, I got carried away…so this also stops pointlessly type
checking popup_widget, as that can never be anything but a GtkTreeMenu.
It still checks for NULL everywhere, which shouldn’t be needed, but (A)
this commit is already too big, & (B) simply removing such checks where
they _seem_ unnecessary causes bad times. I’ll puzzle through that later

testcombo—Don’t set appears-as-list style property

List mode no longer exists, so this had no effect.

combobox: Make wrap-width and grid mode work again

Commit fdc0c6426b6fb751653f3a8536a344f69a6407ac for removing (partly!)
appears-as-list also deleted the code that propagated wrap-width to the
TreeMenu and thus put us into “grid mode”. This restores that code.

And as Benjamin noted, calling check_appearance() here is wrong, so bye.

Add a section about initialization to the migration guide

Just the facts.

Update callers

Adapt all our tests and examples to the new initialization api.

Drop gdk init api

Supporting separately initialized gdk is just not worth it.

Drop arguments from gtk_init

This is our ultimate goal: no more commandline argument handling.
Lots of callers will have to be updated.

Simplify initialization code

Now that gtk_get_option_group and gtk_parse_args are no longer
public, we can get rid of them.

Drop the --gtk-debug and --gtk-no-debug options

We have environment variables that cover this.

Drop the --g-fatal-warnings argument

We don't need it, GLib has an environment variable for this.

Drop the --gtk-modules argument

We don't need it; the environment variable is enough.

Don't export gtk_get_option_group

We want to get rid of commandline option handling in GTK+.
This is a step in that direction.

Stop exporting gtk_parse_args

We want to stop handling commandline arguments, and that is the
sole purpose of this function. So it has to go

Drop gtk_init_with_args

We want to simplify our initialization code and remove all commandline
argument handling from it. The first stop for this is to reduce the
number of gtk_init variants we have.

gdk: Skip GdkVulkanContext methods when building introspection data

We cannot use Vulkan types through introspection anyway.

wayland: Push NULL buffer when hiding a GdkWindow

This is how windows are meant to be hidden as per the wayland
protocol, there's no need to destroy the xdg_surface and other
interfaces.

Also, rename gdk_wayland_window_hide_surface() to clear_surface(),
as that's what it does.

https://bugzilla.gnome.org/show_bug.cgi?id=773686

combobox: Avoid a pointless assignment

Don’t get the active item pointer before the grid/non-grid conditional,
because if we’re in grid mode, we re-get it before selecting it anyway.

combobox: Also preselect active item in grid popup

i.e. when wrap-width > 0. This was only being done for non-grid cases.
So, ComboBoxes in grid mode did not indicate their selection when popped
up and required users to keynav from ‘nothing’ (at the top-left) to the
item they wanted to select. By selecting the active item in advance, now
it’s highlighted & acts as the starting point for keynav around the grid

scale: Really fix documentation of set_digits()

This previously only mentioned its effect on the displayed value, and
even after the previous commit, its rounding of the actual value upon
change still reads like too much of an afterthought. Worse, it wasn’t
mentioned at all in the doc for the @digits parameter. Change this to
emphasise rounding always occurs and the displayed value is secondary.

scale: doc: set_digits doesn’t round retroactively

Whether it should is an open question, but for now, the documentation
should clearly indicate that currently rounding is only applied upon
changes to the value, not to the existing value when ::digits changes.
This is already clear in the doc for the underlying Range::round-digits.

https://bugzilla.gnome.org/show_bug.cgi?id=358970

Scale: Always sync ::digits to Range::round-digits

The documents state that gtk_scale_set_digits() “causes the value of the
adjustment to be rounded off to this number of digits, so the retrieved
value matches the value the user saw.” Note the lack of any condition.

But in fact, if draw-value was false, rounding was disabled on the base
Range, so values that weren’t displayed weren’t rounded. This made the
docs wrong and made an apparently cosmetic detail alter functionality.

Fix by ensuring the number of digits set on Scale is always propagated
along to gtk_range_set_round_digits(), thus rounding to it in all cases
when the value changes, regardless of whether the value is displayed.

This doesn’t address the other idea from Bugzilla: that changing the
number of digits should clamp the _existing_ value if it’s more precise.
This contradicts digits docs in the base Range, but the above from Scale
can be read as implying it’ll happen. For now, that’s an open question.

https://bugzilla.gnome.org/show_bug.cgi?id=358970

combobox: Work around popup handler altering model

GtkFileChooserButton installs a handler for the popped-up signal, which
refilters the menu, in order to hide the “(None)” item from the popup
if it was previously selected in the ComboBox. This oddity means that:

 • Until recently, this item would be selected in the menu shell, which
   would then be popped up and change the selection away from that item.
   This was therefore redundant (more on which below!) but benign.

 • After the patch for https://bugzilla.gnome.org/show_bug.cgi?id=771242
   however, this causes a critical assertion fail, as now we stash the
   originally selected item in a pointer so that it can be selected only
   after realisation/popup – but by that stage, the model has just been
   refiltered and the previous pointer no longer refers to a valid item.

This commit works around this problem by, after popping up the menu,
getting the active item again, in case a popped-up handler has gone and
invalidated the pointer to the active item that we saved before popup.

If a handler does this, everything done to find/use the original item is
pointless. But this avoids the ugly critical in FileChooserButton, while
not harming every other ComboBox that doesn’t mess with its model while
popping up (hopefully the vast majority), and it’s very difficult to
imagine a way to check if the active item is /going to/ be hidden later)

combobox: Don’t select active item if it’s hidden

I hope no one ever actually brings such a silly item into this world,
but this achieves symmetry with the similar checks immediately after.

combobox: Move variables into narrowest scopes

Update Slovak translation

vulkan: Fix copy/paste error

vulkan: Add compiled shaders

I forgot to include them when adding the shaders.

vulkan: Compute right size for outset shadow

We were emitting the box and not the shadow box as the rectangle. So we
didn't draw any shadows. Oops.

css: Remove deprecated number-as-pixels compatibility

Previously, for compatibility with GTK 3.0, we allowed specifying
numbers without units and interpreted them as pixels, even when the CSS
specification didn't.

Remove that now that we can break API.